Skip to content

fix failing installation for tests #862

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

japj
Copy link

@japj japj commented Apr 5, 2025

This ONLY fixes the failing installation of test step in Github CI.

The single threaded tests are passing, but the multi-threaded tests are (still) failing in the CI job.

Copy link

netlify bot commented Apr 5, 2025

Deploy Preview for ffmpegwasm canceled.

Name Link
🔨 Latest commit daf8ae2
🔍 Latest deploy log https://app.netlify.com/sites/ffmpegwasm/deploys/67f19515e8f19d0008aea604

@japj
Copy link
Author

japj commented Apr 5, 2025

I replaced http-server with serve.
The http-server custom headers functionality is not yet in a released version of http-server

So at least the multi-threaded tests start executing locally again, but still fail:

Screenshot 2025-04-05 at 22 21 17
RuntimeError: Out of bounds memory access (evaluating 'Module["_ffmpeg"](args.length,stringsToPtr(args))')<?>.wasm-function[1894]@[wasm code]
<?>.wasm-function[1725]@[wasm code]
<?>.wasm-function[6106]@[wasm code]
<?>.wasm-function[810]@[wasm code]
<?>.wasm-function[4106]@[wasm code]
<?>.wasm-function[1719]@[wasm code]
<?>.wasm-function[8312]@[wasm code]
<?>.wasm-function[85]@[wasm code]
<?>.wasm-function[18631]@[wasm code]
exec@http://localhost:3000/packages/core-mt/dist/umd/ffmpeg-core.js:8:1229
@http://localhost:3000/tests/ffmpeg-core.test.js:51:21

not sure why this is failing for a different reason on CI though: https://github.com/japj/ffmpeg.wasm/actions/runs/14285449729/job/40040006250 Screenshot 2025-04-05 at 22 31 39

@japj
Copy link
Author

japj commented Apr 5, 2025

fixed the 301 issue with serve

The multithreaded tests still fail, both locally and in CI.
But I don't know if I can solve that, since I have no experience debugging web assembly code.

@japj
Copy link
Author

japj commented Apr 6, 2025

I now see that there is another related pullrequest trying to solve failing install/test core-mt test failures at #842

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant